Skip to content

BUG: use hash-function which takes nans correctly into account for ExtensionDtype #42135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 22, 2021

Conversation

realead
Copy link
Contributor

@realead realead commented Jun 19, 2021

  • tests added / passed
  • Ensure all linting tests pass, see here for how to run them

pandas/tests/arrays/sparse/test_dtype.py test_hash_equal fails on Python3.10 (see #41940). This is due to the fact, that since Python3.10 different nan-objects have different hashes.

In #41988 a hash function with the old pre-Python3.10 behavior was introduced. This function is used in ExtensionDtype to keep the correct behavior also with Python3.10.

@lithomas1 lithomas1 added hashing hash_pandas_object ExtensionArray Extending pandas with custom dtypes or arrays. Python 3.10 labels Jun 20, 2021
@realead realead force-pushed the right_hash_and_equal branch from 93fc438 to 5fd6366 Compare June 21, 2021 13:20
@jreback jreback added this to the 1.3 milestone Jun 22, 2021
@jreback jreback merged commit 3bb31da into pandas-dev:master Jun 22, 2021
@jreback
Copy link
Contributor

jreback commented Jun 22, 2021

thanks @realead

@jreback
Copy link
Contributor

jreback commented Jun 22, 2021

@meeseeksdev backport 1.3.x

@lumberbot-app
Copy link

lumberbot-app bot commented Jun 22, 2021

Something went wrong ... Please have a look at my logs.

simonjayhawkins pushed a commit that referenced this pull request Jun 22, 2021
… into account for ExtensionDtype (#42183)

Co-authored-by: realead <[email protected]>
JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Jul 3, 2021
@lithomas1 lithomas1 mentioned this pull request Jul 13, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ExtensionArray Extending pandas with custom dtypes or arrays. hashing hash_pandas_object Python 3.10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants